Skip to content

Conversation

@erwango
Copy link
Member

@erwango erwango commented Mar 11, 2019

Following new shield introduction in #14057, it has been highlighted
there was no codeowner for boards/shields/.
Assign erwango as codeowner.

Signed-off-by: Erwan Gouriou [email protected]

Following new shield introduction in zephyrproject-rtos#14057, it has been highlighted
there was no codeowner for boards/shields/.
Assign erwango as codeowner.

Signed-off-by: Erwan Gouriou <[email protected]>
@codecov-io
Copy link

Codecov Report

Merging #14249 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #14249   +/-   ##
=======================================
  Coverage   52.02%   52.02%           
=======================================
  Files         307      307           
  Lines       45484    45484           
  Branches    10535    10535           
=======================================
  Hits        23662    23662           
  Misses      17030    17030           
  Partials     4792     4792

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7967829...76283c9. Read the comment docs.

@mike-scott mike-scott self-requested a review March 11, 2019 15:20
Copy link
Contributor

@mike-scott mike-scott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mike-scott mike-scott requested review from galak and nashif March 11, 2019 15:21
@mike-scott mike-scott added this to the v1.14.0 milestone Mar 11, 2019
@nashif nashif merged commit 7d168db into zephyrproject-rtos:master Mar 11, 2019
@erwango erwango deleted the codeowner_shields branch September 3, 2019 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants